Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove obsolete l10n strings (Fix #15496) #15770

Merged
merged 3 commits into from
Dec 19, 2024
Merged

Remove obsolete l10n strings (Fix #15496) #15770

merged 3 commits into from
Dec 19, 2024

Conversation

stephaniehobson
Copy link
Contributor

One-line summary

L10n maintenance

Significant changes and points to review

  • removed obsolete strings which have expired
  • discovered CTD templates were referencing a string ID that had been removed, took the translations off prod and hard coded them like the other strings in the CTD templates

Issue / Bugzilla link

Fix #15496

Testing

Do a make preflight to make sure you have the most recent translations.

http://localhost:8000/fr/firefox/all/
http://localhost:8000/fr/firefox/features/adblocker/
http://localhost:8000/fr/firefox/135.0a1/whatsnew/
http://localhost:8000/fr/newsletter/firefox/

http://localhost:8000/de/firefox/challenge-the-default/
http://localhost:8000/fr/firefox/challenge-the-default/
http://localhost:8000/es-ES/firefox/challenge-the-default/
http://localhost:8000/it/firefox/challenge-the-default/
http://localhost:8000/pl/firefox/challenge-the-default/

@stephaniehobson stephaniehobson requested a review from a team as a code owner December 18, 2024 05:43
@stephaniehobson stephaniehobson added L10N Needs Review Awaiting code review Review: XS Code review time: up to 30min labels Dec 18, 2024
@wen-2018 wen-2018 self-assigned this Dec 19, 2024
@wen-2018 wen-2018 removed the Needs Review Awaiting code review label Dec 19, 2024
Copy link
Collaborator

@wen-2018 wen-2018 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a minor clean up needed. r+wc

@@ -29,14 +26,8 @@ firefox-all-down-arrow = Choose from the list below

# Used as an accessible label for a help button. The text is replaced with a "?" icon.
firefox-all-get-help = Get help
# Obsolete string (expires 2024-10-30)
firefox-all-browser = Browser:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch, thanks!

@stephaniehobson stephaniehobson merged commit 12806d5 into main Dec 19, 2024
4 checks passed
@stephaniehobson stephaniehobson deleted the bot-l10n branch December 19, 2024 19:11
Copy link

codecov bot commented Dec 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.07%. Comparing base (a2eb92c) to head (6a81c53).
Report is 5 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #15770   +/-   ##
=======================================
  Coverage   79.07%   79.07%           
=======================================
  Files         160      160           
  Lines        8359     8360    +1     
=======================================
+ Hits         6610     6611    +1     
  Misses       1749     1749           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
L10N Review: XS Code review time: up to 30min
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[bot] Remove obsolete l10n strings and ftl files
2 participants